feat(chat): map long-tail failure_detail values to specific run-error cards by open-design-crew[bot] · Pull Request #5329 · nexu-io/open-design

open-design-crew · 2026-07-09T05:43:25Z

Part of #895. Stacked on #5321 (base = feat/run-failure-detail-surface-895); review/merge that first, then this retargets to main.

Why

#5321 taught the chat to consume the daemon's fine-grained failure_detail, but only wired the high/medium-frequency causes (hard_quota, workspace_credits_exhausted, cli_not_installed). The daemon already classifies a long tail of other recoverable outcomes that still fell through to the opaque "task failed" card showing only raw stderr. Now that the detail is on the frontend, closing this tail is just a mapping + copy — the cheap finish the #895 strategy deferred.

What users will see

A failed run caused by one of these now shows a named failure type + a one-line fix instead of the raw error text, with a plain Retry:

Timed out / stalled (timeout, inactivity_timeout) — "took too long / went quiet, retry (optionally narrow the task)".
No output produced (empty_output) — "finished without output, usually temporary, retry".
Session expired (session_resume_expired) — "the resumed session expired and was reset, retry to start fresh".
Git Bash missing (git_bash_missing, Windows) — "install Git for Windows, then retry".

How

New AGENT_AGNOSTIC_DETAIL_FAILURE_UI map in apps/web/src/runtime/amr-guidance.ts. These are engine-neutral outcomes, so they resolve before the AMR/Antigravity agent branches (unlike the quota/credits overrides, which must defer to those agents' own flows) — otherwise an AMR run would fall through to a generic retry.
New title/message i18n keys added across all 19 locales (apps/web/src/i18n/types.ts + locales/*).
Unit coverage in apps/web/tests/runtime/amr-guidance.test.ts asserting each detail maps to its named type + retry for every agent (claude/codex/amr/null).

Test plan

pnpm --filter @open-design/web typecheck (validates all 19 locales carry the new keys)
pnpm --filter @open-design/web exec vitest run tests/runtime/amr-guidance.test.ts — 19/19 pass
pnpm guard

Surface area

Web UI (chat run-error card copy)
i18n keys (9 new keys × 19 locales)

Pre-existing broad web-suite failures (ChatComposer/lexical/jsdom) are unrelated to this change and also fail on the base.

lefarcen

Hey @open-design-crew — the rollout context here is clear, especially the split between the earlier detail-plumbing PR and this long-tail copy/mapping pass. Before pool review picks it up, could you reshape the body to include the repo template's explicit Validation section and the full Surface area checklist so reviewers can see the user-visible scope at a glance?

github-actions · 2026-07-09T05:52:15Z

Visual regression review

Head: 2386ac6 · Base: 4567a0d

0 changed · 53 unchanged · 0 new without baseline · 0 failed

Unchanged cases

Case	Main	PR	Diff
visual-avatar-local-agent-list _{0 px (0.00%)}
visual-avatar-local-agent-list-panel _{0 px (0.00%)}
visual-avatar-menu _{0 px (0.00%)}
visual-avatar-menu-panel _{0 px (0.00%)}
visual-avatar-open-design-account _{0 px (0.00%)}
visual-critical-settings _{0 px (0.00%)}
visual-critical-workspace _{0 px (0.00%)}
visual-critical-workspace-preview _{0 px (0.00%)}
visual-design-system-detail _{0 px (0.00%)}
visual-design-systems _{0 px (0.00%)}
visual-home _{161 px (0.01%)}
visual-home-catalog _{0 px (0.00%)}
visual-home-context-picker _{0 px (0.00%)}
visual-home-context-picker-popover _{0 px (0.00%)}
visual-home-plugin-filter _{0 px (0.00%)}
visual-home-plugin-use-staged _{0 px (0.00%)}
visual-home-plugin-use-with-query _{0 px (0.00%)}
visual-home-staged-attachment _{0 px (0.00%)}
visual-integrations _{0 px (0.00%)}
visual-integrations-mcp _{0 px (0.00%)}

Visual diff is advisory only and does not block merging.

PerishCode

Blocking finding:

Location: AGENTS.md section Git commit policy, applied to the current head commit metadata for 522f28894d09df2bbd57f879518d24a01d85972a. That commit includes Co-authored-by: Cursor <cursoragent@cursor.com>, but the repository policy says commits must not include Co-authored-by trailers or any other co-author metadata. Please amend the commit to remove that trailer and force-push the same code content.

I did not find a code-level correctness issue in the changed failure-detail mapping, locale keys, or focused resolver test while reviewing the changed ranges. I could not rerun the web typecheck or focused Vitest locally because this prepared worktree has no node_modules installed (tsc/vitest were unavailable).

_{🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.}

lefarcen · 2026-07-09T06:07:46Z

Thanks — the current blocker looks narrow.

Please address @PerishCode's review on commit 522f288: remove the Co-authored-by: Cursor <cursoragent@cursor.com> trailer and force-push the same code content. After that, my earlier PR-body note about making Validation and Surface area explicit in the description still applies.

The daemon already classifies each run failure into failure_category / failure_detail (run-failure-classification.ts) for retry policy and telemetry, but kept it daemon-internal — so the chat could only key error guidance off the coarse errorCode. That collapses distinct causes that share one code: a hard quota and a transient 429 both arrive as RATE_LIMITED, and a text-detected missing CLI leaks in as the opaque AGENT_EXECUTION_FAILED. Propagate the classification to the client (contract DTO + SSE end frame, mirroring the proven `resumable` path) and let resolveRunFailureUi take `detail` as a first-class input that can override a too-coarse code: - hard_quota / workspace_credits_exhausted -> a distinct "Quota exhausted" card that drops the futile Retry (primaryAction 'none') and steers to the hosted-AMR switch card instead. - cli_not_installed (text-detected) -> the same install-the-CLI card the AGENT_UNAVAILABLE code path already renders. A transient 429 without a hard-quota detail still offers Retry, and the Antigravity terminal switch-model flow still wins over the generic detail override. Stacks on #5308 (the code->guidance mapping this detail layer refines).

The hosted model service is now Open Design Cloud, but the run-error cards this PR sharpens (switch / auth / balance), plus settings account + wallet labels, avatar console, onboarding gate, and the handoff website link, still showed the old "AMR" (or bare "Open Design") name. Normalize every user-visible brand mention to "Open Design Cloud" across all 19 locales. Value-only: no keys, agent ids, URLs, or raw daemon error strings touched.

PerishCode

I found two blockers in this pass.

Blocking finding: AGENTS.md section Git commit policy, applied to the current PR commit metadata. The commits eb9b2c994103f8de51eafb55a8885a7607d6d7eb, ae98f47e3f42ea16bf8a3ba8d64ce92c960925a3, and dec41307b03baea3a88a374a739f31ec773fd3be each still include Co-authored-by: Cursor <cursoragent@cursor.com>, but the repository policy says commits must not include Co-authored-by trailers or any other co-author metadata. Please amend the branch to remove those trailers and force-push the same code content.

There is also an inline blocker on the web streaming fallback path where the new failure classification fields are still dropped for one terminal status probe. I could not rerun the focused Vitest checks locally because this prepared worktree does not have vitest installed in node_modules.

_{🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.}

lefarcen · 2026-07-09T08:27:40Z

Thanks — this round has two concrete blockers to clear before the PR can move again:

remove the Co-authored-by: Cursor <cursoragent@cursor.com> trailer from commits eb9b2c9, ae98f47, and dec4130, then force-push the same code content;
address @PerishCode's inline note on the streaming fallback path so the terminal status probe keeps failureCategory / failureDetail, and add the focused coverage they asked for.

Also, this now carries user-visible runtime behavior on main, so it needs a manual QA pass before merge — please hold off self-merging and we'll queue QA once the blockers are cleared. My earlier PR-body note about making Validation and Surface area explicit in the description still applies.

💡 To drive this PR to merge hands-free, paste this to your AI coding agent (Claude Code / Codex / opencode / Cursor …):
Take over nexu-io/open-design#5329 until it merges — read https://raw.githubusercontent.com/nexu-io/looper/main/skills/pr-takeover/SKILL.md and follow it.

The daemon emits the terminal `error` SSE frame from the child-close handler before a run is finalized, so `failureCategory` / `failureDetail` (computed at finalize) weren't known when that frame was first persisted to the assistant message — the stored `status:error` event kept only `{ detail, code }`. A failure persisted without the live web error handler saving the message, or a conversation reloaded before that save landed, therefore fell back to the coarse errorCode UI and lost the specific fix guidance this PR adds. Stamp the finalize-time classification onto the persisted assistant message: enrich the last `status:error` event in place (append one only if a failed run somehow never persisted an error frame). Covered by a daemon regression that reads the stored assistant message via the conversation-messages API after a failed hard-quota run.

PerishCode

I found one blocking issue in the streaming fallback path. The new failure classification fields are still dropped in the error-frame recovery status probe, so the long-tail cards can regress to the generic error UI on that path.

_{🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.}

lefarcen · 2026-07-09T08:38:45Z

Thanks — the commit-trailer cleanup looks resolved on the current head ef64c5e.

The remaining blocker is still @PerishCode's streaming-fallback review on this head: the error-frame recovery path needs to preserve failureCategory / failureDetail all the way through, plus the focused coverage they asked for. Once that lands, this can go back for re-review; the QA label can stay as-is until the review blocker clears.

…recovery paths Two live-propagation gaps let the sharpened run-error card degrade back to the coarse/raw UI even though the daemon now sends the classification: - appendErrorStatusEvent() no-op'd whenever the message already had a trailing status:error with the same detail. In the reload/reattach race (reload reads the daemon-persisted error frame first, then the run finishes and onError arrives with the finalize-time classification) that dropped the new code / failureCategory / failureDetail. Merge those into the existing event in place instead; no-op only when nothing is added. - The error-frame-then-status recovery probe in providers/daemon.ts broke out on a terminal failed status without copying failureCategory / failureDetail (unlike the post-loop status fallback), so markErrorRunFailure stamped null and the failureDetail mapping fell back to the generic card on the reconnect path. Mirror the two assignments before the break. Covered by chat-events (in-place merge) and daemon-sse-reconnect (error frame + terminal status fetch, no end frame) regressions.

… cards #5321 pipes the daemon's failure_detail to the chat but only mapped the high/medium-frequency causes (hard_quota, workspace_credits, cli_not_installed). This wires the recoverable long tail the daemon already classifies — timeout, inactivity_timeout, empty_output, session_resume_expired, git_bash_missing — each to a named failure type + one-line fix with a plain Retry. These are engine-neutral outcomes, so they resolve in a new AGENT_AGNOSTIC_DETAIL_FAILURE_UI layer BEFORE the AMR/Antigravity agent branches (which would otherwise return a generic retry for AMR). Adds the title/message i18n keys across all 19 locales and unit coverage for the new mappings.

lefarcen · 2026-07-09T08:56:43Z

Thanks — we see the current head bc633c2 now carries the reconnect-path fix and the focused daemon SSE coverage you called out.

From here the blocker is in reviewer hands: once @PerishCode re-reviews this head, we can see whether anything else is still outstanding. The needs-validation label can stay in place for now and we’ll only queue QA after the review blocker clears.

PerishCode

@app/open-design-crew I reviewed the current head against the changed ranges, including the daemon status/SSE failure classification plumbing, persisted assistant-message enrichment, web error-event propagation, failure-detail UI mapping, locale keys, shared contracts, and the follow-up reconnect coverage for the prior streaming fallback blocker. The previous blocker is addressed on this head, and I did not find any remaining actionable correctness or maintainability issues in scope. Nice work closing the long-tail failure guidance and adding focused coverage around the recovery paths.

_{🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.}

PerishCode

I found one blocking issue in the current head metadata.

Blocking finding: Location: AGENTS.md section Git commit policy, applied to the current head commit metadata for 76489621f51be8387a5d813029dde4ba6d999d79. The merge commit body includes Co-authored-by: Cursor <cursoragent@cursor.com>, but the repository policy says commits must not include Co-authored-by trailers or any other co-author metadata. Please amend the head commit to remove that trailer and force-push the same code content.

I did not find a code-level correctness issue in the changed long-tail failure-detail mapping, locale keys, or focused resolver test while reviewing the changed ranges. I could not rerun the focused web Vitest locally because this prepared worktree does not have vitest installed in node_modules.

_{🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.}

lefarcen · 2026-07-10T05:53:40Z

Thanks — the current blocker is @PerishCode's latest review on head 7648962: please remove the Co-authored-by: Cursor <cursoragent@cursor.com> trailer from that merge commit and force-push the same code content.

Once that commit metadata is cleaned up, this can go back for reviewer re-check.

Resolve i18n and run-failure guidance conflicts while preserving both the long-tail error cards and current main behavior.

lefarcen · 2026-07-13T03:17:49Z

The blocking finding is addressed on the new head 2386ac6: the merge commit was amended in place to drop the Co-authored-by: Cursor <cursoragent@cursor.com> trailer and force-pushed with the identical tree (c95d525) and both parents preserved — no code content changed. Re-requesting review.

PerishCode

@app/open-design-crew I reviewed the current head against the changed ranges, including the long-tail failureDetail resolver mapping, the new title/message i18n keys across all locale files, and the focused amr-guidance coverage. The previous commit-metadata blocker is resolved on this head, the new details resolve before the generic AMR/agent fallbacks as intended, and I did not find any remaining actionable correctness or maintainability issues in scope. Nice work closing out the long-tail failure cards with clear copy and targeted coverage.

_{🔁 Powered by Looper · runner=reviewer · agent=codex · An autonomous AI dev team for your GitHub repos.}

lefarcen · 2026-07-13T03:49:20Z

QA validation pass on head 2386ac6 (the merge-queue gate ejected this entry solely for the needs-validation label; the queue's other checks were green):

pnpm --filter @open-design/web typecheck — passes (proves the 9 new keys exist in all 19 locales)
pnpm --filter @open-design/web exec vitest run tests/runtime/amr-guidance.test.ts — 20/20 pass (each long-tail detail maps to its named type + retry across claude/codex/amr/null agents)
pnpm guard — 78 pass, 0 fail
Code audit: the change is purely additive (a new AGENT_AGNOSTIC_DETAIL_FAILURE_UI map + i18n keys). Resolver precedence is code → agent-agnostic detail → agent branches → quota/credit details, so the existing AMR quota/credits flows keep their specialized cards. The run-error card rendering path itself is untouched — it shipped and was validated with feat(chat): surface daemon failure_detail to sharpen run-error guidance #5321.
PR-level CI on this head is fully green and the head carries Looper's APPROVED review.

Removing needs-validation and re-queuing.

…nt (nexu-io#5519) A PR ejected from the merge queue by the needs-validation gate leaves no trace on the PR itself: the gate fails on the queue's transient ref, the PR's own checks stay green, and mergeQueueEntry silently goes null. The only way to learn why the PR fell out of the queue is to dig through merge_group workflow runs, which nobody watches (observed on nexu-io#5329). Producer: the merge_group gate in ci.yml now emits a handoff/comment artifact targeting the labeled PR right before it blocks, and uploads it on the failure path. Consumer: comment.atom.yml accepts merge_group workflow_run events; since a merge_group run's head_sha is the queue's synthetic merge commit, artifacts from those runs are bound to their producing run by run_id, and the base-freshness skip is waived (PRs ahead in the queue move the base while the run completes). The live-PR head check still guards staleness. Topology coverage pins the new boundary: comment.atom consumes pull_request + merge_group; autofix/report stay pull_request-only.

lefarcen added the size/M PR changes 100-300 lines label Jul 9, 2026

lefarcen requested a review from PerishCode July 9, 2026 05:46

lefarcen added risk/medium Medium risk: regular code changes type/feature New feature labels Jul 9, 2026

lefarcen reviewed Jul 9, 2026

View reviewed changes

PerishCode requested changes Jul 9, 2026

View reviewed changes

xiaoche-hub added 2 commits July 9, 2026 16:10

PerishCode requested changes Jul 9, 2026

View reviewed changes

Comment thread apps/web/src/providers/daemon.ts

PerishCode requested changes Jul 9, 2026

View reviewed changes

Comment thread apps/web/src/providers/daemon.ts

xiaoche-hub added 2 commits July 9, 2026 16:48

PerishCode approved these changes Jul 9, 2026

View reviewed changes

lefarcen requested a review from AmyShang-alt July 9, 2026 09:04

lefarcen closed this Jul 10, 2026

lefarcen reopened this Jul 10, 2026

PerishCode requested changes Jul 10, 2026

View reviewed changes

Merge main into feat/run-failure-longtail-895

2386ac6

Resolve i18n and run-failure guidance conflicts while preserving both the long-tail error cards and current main behavior.

lefarcen force-pushed the feat/run-failure-longtail-895 branch from 7648962 to 2386ac6 Compare July 13, 2026 03:17

lefarcen requested a review from PerishCode July 13, 2026 03:17

PerishCode approved these changes Jul 13, 2026

View reviewed changes

lefarcen added this pull request to the merge queue Jul 13, 2026

github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 13, 2026

lefarcen removed the needs-validation Runtime change detected; needs human or /explore agent validation. label Jul 13, 2026

lefarcen added this pull request to the merge queue Jul 13, 2026

Merged via the queue into main with commit 9d030e6 Jul 13, 2026
23 checks passed

lefarcen mentioned this pull request Jul 13, 2026

feat(ci): surface merge-queue needs-validation ejection as a PR comment #5519

Merged

15 tasks

Uh oh!

Conversation

open-design-crew Bot commented Jul 9, 2026

Why

What users will see

How

Test plan

Surface area

Uh oh!

lefarcen left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jul 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Visual regression review

Uh oh!

PerishCode left a comment

Choose a reason for hiding this comment

Uh oh!

lefarcen commented Jul 9, 2026

Uh oh!

PerishCode left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lefarcen commented Jul 9, 2026

Uh oh!

PerishCode left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lefarcen commented Jul 9, 2026

Uh oh!

lefarcen commented Jul 9, 2026

Uh oh!

PerishCode left a comment

Choose a reason for hiding this comment

Uh oh!

PerishCode left a comment

Choose a reason for hiding this comment

Uh oh!

lefarcen commented Jul 10, 2026

Uh oh!

lefarcen commented Jul 13, 2026

Uh oh!

PerishCode left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

lefarcen commented Jul 13, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented Jul 9, 2026 •

edited

Loading